Skip to content

feat: Map Selected Identity to Emailsha256#29

Merged
BrandonStalnaker merged 2 commits into
developmentfrom
feat/SQDSDKS-7631-Map-Selected-Identity-to-Emailsha256
Aug 20, 2025
Merged

feat: Map Selected Identity to Emailsha256#29
BrandonStalnaker merged 2 commits into
developmentfrom
feat/SQDSDKS-7631-Map-Selected-Identity-to-Emailsha256

Conversation

@BrandonStalnaker

Copy link
Copy Markdown
Collaborator

Summary

  • Updated the logic to use the selected identity as the emailsha256 identity for Rokt. Defaults to 'other' in case this were to go live before the backend change.

Testing Plan

  • Was this tested locally? If not, explain why.
  • Tested in the sample app and through unit tests

Reference Issue (For mParticle employees only. Ignore if you are an outside contributor)

@BrandonStalnaker BrandonStalnaker self-assigned this Aug 19, 2025
@BrandonStalnaker BrandonStalnaker requested a review from a team as a code owner August 19, 2025 19:32
Comment thread mParticle-Rokt/MPKitRokt.m Outdated
Comment thread mParticle-Rokt/MPKitRokt.m Outdated
Comment thread mParticle_RoktTests/mParticle_RoktTests.m

@nickolas-dimitrakas nickolas-dimitrakas left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@denischilik denischilik self-requested a review August 20, 2025 18:47

@denischilik denischilik left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@BrandonStalnaker BrandonStalnaker merged commit ee4e7fc into development Aug 20, 2025
5 checks passed
@BrandonStalnaker BrandonStalnaker deleted the feat/SQDSDKS-7631-Map-Selected-Identity-to-Emailsha256 branch August 20, 2025 18:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants